luci-app-acl: unset read and write before acl set

If the setting in the view is set to `denied`, only the read list option
is deleted. This is not correct. The write list option must also be
deleted. To ensure that the correct configuration is saved, the write
and read list options are always deleted beforehand and then rewritten.

Signed-off-by: Florian Eckert <[email protected]>

diff --git a/applications/luci-app-acl/htdocs/luci-static/resources/view/system/acl.js b/applications/luci-app-acl/htdocs/luci-static/resources/view/system/acl.js
index 618facd6e64c45e9d4b2c8ed59319fa1ecea0430..8d1086f754b67cc216f3d76be4ad1679ec3b0ab1 100644 (file)
--- a/applications/luci-app-acl/htdocs/luci-static/resources/view/system/acl.js
+++ b/applications/luci-app-acl/htdocs/luci-static/resources/view/system/acl.js
@@ -146,6 +146,9 @@ var cbiACLSelect = form.Value.extend({
        },
 
        write: function(section_id, value) {
+               uci.unset('rpcd', section_id, 'read');
+               uci.unset('rpcd', section_id, 'write');
+
                if (L.isObject(value) && Array.isArray(value.read))
                        uci.set('rpcd', section_id, 'read', value.read);